Skip to content

StreamableHttp transport - session management #345

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 6 commits into from
Apr 17, 2025
Merged

StreamableHttp transport - session management #345

merged 6 commits into from
Apr 17, 2025

Conversation

ihrpr
Copy link
Contributor

@ihrpr ihrpr commented Apr 16, 2025 •
edited
Loading

With StreamableHttp transport, servers can establish stateful sessions as well as they can operate in stateless mode.

  • adding utility to check isInitializeRequest
  • adding utility to check isInitializedNotification
  • introducing oncessioninitialized to avoid race conditions
  • adding simple integration tests for stateless and stateful sessions
  • adding session termination to the client

@smurf28
Copy link

smurf28 commented Apr 16, 2025

Can stateless services be debugged in Inspector?

@ihrpr ihrpr changed the title Ihrpr/session management StreamableHttp transport - session management Apr 16, 2025
@ihrpr
Copy link
Contributor Author

ihrpr commented Apr 16, 2025

Can stateless services be debugged in Inspector?

@smurf28 yes, all of the server/transport types should be debuggable in Inspector.
I don't believe we merged Inspector changes yet to support streamable http yet -- coming soon

@ihrpr ihrpr marked this pull request as ready for review April 16, 2025 12:05
@ihrpr ihrpr mentioned this pull request Apr 16, 2025
Merged
jspahrsummers
jspahrsummers approved these changes Apr 17, 2025
View reviewed changes
@ajgray-stripe ajgray-stripe mentioned this pull request Apr 17, 2025
Closed
@ihrpr ihrpr merged commit 4e3ddf4 into main Apr 17, 2025
4 checks passed
@ihrpr ihrpr deleted the ihrpr/session_management branch April 17, 2025 16:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jspahrsummers jspahrsummers jspahrsummers approved these changes

@jerome3o-anthropic jerome3o-anthropic Awaiting requested review from jerome3o-anthropic

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ihrpr @smurf28 @jspahrsummers